Skip to content

Restore sidebar transition when switching from/to small screen/"sticky" sidebar #123

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 11, 2017

Conversation

fk
Copy link
Contributor

@fk fk commented Oct 10, 2017

No description provided.

Restores the sidebar transition when switching from and to small screen/"sticky" sidebar.
@reactjs-bot
Copy link

Deploy preview ready!

Built with commit 3a238c8

https://deploy-preview-123--reactjs.netlify.com

Copy link
Contributor

@bvaughn bvaughn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm actually not sure if we want this transition 😁 Although transform was totally mis-typed before which was unintentional. I'm not sure it looks better with it enabled though.

What do you think, @joecritch?

@fk
Copy link
Contributor Author

fk commented Oct 10, 2017

I'm actually not sure if we want this transition 😁

Yeah, actually asked that myself, too. 😉
FWIW I think it feels OK when going from small to medium, but not so much the other way around.

@bvaughn
Copy link
Contributor

bvaughn commented Oct 10, 2017

We should definitely either fix this property name or remove it entirely, I'm just not sure which. (Leaning towards removing entirely but will see if @joecritch agrees)

@joecritch
Copy link
Contributor

@bvaughn Yeah, no transitions when resizing are intentional :)

@bvaughn
Copy link
Contributor

bvaughn commented Oct 11, 2017

So the typo (tranform) should be fixed (to transform) and you like the way that looks afterward? 😄 Okedoke.

@bvaughn bvaughn merged commit e85d9fc into reactjs:master Oct 11, 2017
@joecritch
Copy link
Contributor

@bvaughn Sorry, I meant the transitions weren’t intentional. But no sweat, I’ll check out master tomorrow to see how it looks ☺️👍

@bvaughn
Copy link
Contributor

bvaughn commented Oct 11, 2017

Please feel free to change it however you'd like Joe (or just shoot me a Hangouts message with clarification and I'll make the change!) 😄

BetterZxx pushed a commit to BetterZxx/react.dev that referenced this pull request Mar 21, 2023
…s#123)

* 翻译了 effect hook 的前半部分

* hooks-effect 翻译完成

* Update hooks-effect.md

修改一些问题

* Update hooks-effect.md

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update hooks-effect.md

* Update hooks-effect.md

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update hooks-effect.md

quota

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update hooks-effect.md

* Update hooks-effect.md

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update hooks-effect.md

* docs:modify hooks-effect doc

* Update hooks-effect.md

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update content/docs/hooks-effect.md

Co-Authored-By: Crazydogs <crazydogazrael@gmail.com>

* Update hooks-effect.md

* Update hooks-effect.md

* Update hooks-effect.md

* Update hooks-effect.md

* Update hooks-effect.md

* Update hooks-effect.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants